-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SPT-1998 Добавление таргета для моков #129
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 5.0.0 #129 +/- ##
=======================================
Coverage 91.42% 91.42%
=======================================
Files 82 82
Lines 1259 1259
=======================================
Hits 1151 1151
Misses 108 108
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
7f72987
to
65734f5
Compare
8ccb83b
to
c927a83
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Здесь или в следующем ПР можно добавить инструкцию небольшую для нового таргета NodeKitMock.
Вроде моки публичных нод есть, но однозначного понятия нет как протестить какой-нибудь метод сервиса, например
func postDeactivate() -> Observer<Void> {
return builder
.route(.post, .deactivate)
.build()
.process()
}
(Да это пример на старом NodeKit, он нужен для пояснения концепции использования)
builder у нас создает chain, цепочку нод.
URLChainsbuilder остался в этой версии, но теперь возвращает AsyncNode или AsyncStreamNode. Однако мока на него нет.
Сервисы как правило реализуют группу запросов, потому чтобы не создавать ручками в проекте мок URLChainsbuilder, лучше его включить в новый подпакет.
65734f5
to
7110bb1
Compare
c927a83
to
babfb15
Compare
7110bb1
to
c5e2ee8
Compare
babfb15
to
77a98d8
Compare
Что сделано